Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update comment blocks. #879

Closed
wants to merge 1 commit into from
Closed

Update comment blocks. #879

wants to merge 1 commit into from

Conversation

yuri-kiss
Copy link
Contributor

Adds matrix comments, and also color comments.
sadly since C-Blocks don't work i could not add them 😭.

@GarboMuffin
Copy link
Member

@LilyMakesThings this is your extension

do you agree with this?

@LilyMakesThings
Copy link
Contributor

LilyMakesThings commented Aug 22, 2023

I don't like this. I'd prefer people to use a coupler for these values.

Adding a comment to a colour or matrix value is so niche that, for the one person who is going to do that, they should just use the extra block to do that if it keeps this palette to a minimum.

@NexusKitten

This comment was marked as spam.

@GarboMuffin
Copy link
Member

related: #757

@GarboMuffin
Copy link
Member

Similar blocks are in couplers now

@yuri-kiss yuri-kiss deleted the comment-blocks-update branch August 23, 2023 17:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants